Skip to content

fix: handle missing git command when getting local version#557

Merged
luislhl merged 1 commit intodevfrom
fix/ci-docker-build
Apr 17, 2023
Merged

fix: handle missing git command when getting local version#557
luislhl merged 1 commit intodevfrom
fix/ci-docker-build

Conversation

@luislhl
Copy link
Copy Markdown
Contributor

@luislhl luislhl commented Apr 17, 2023

Acceptance Criteria

We shouldn't get a FileNotFoundError if there is no git command available when trying to run it to build the local version

An example of such case happened in https://github.com/HathorNetwork/hathor-core/actions/runs/4721185503/jobs/8374162049

@luislhl luislhl self-assigned this Apr 17, 2023
@luislhl luislhl force-pushed the fix/ci-docker-build branch from 0c5f356 to c01acd7 Compare April 17, 2023 17:11
@luislhl luislhl requested review from glevco and removed request for msbrogli April 17, 2023 17:18
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 2023

Codecov Report

Merging #557 (c01acd7) into dev (7573c6f) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #557      +/-   ##
==========================================
+ Coverage   83.25%   83.26%   +0.01%     
==========================================
  Files         212      212              
  Lines       18723    18723              
  Branches     2595     2595              
==========================================
+ Hits        15588    15590       +2     
+ Misses       2567     2566       -1     
+ Partials      568      567       -1     
Impacted Files Coverage Δ
hathor/version.py 85.29% <0.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@luislhl luislhl merged commit 07fc529 into dev Apr 17, 2023
@luislhl luislhl deleted the fix/ci-docker-build branch April 17, 2023 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants